Components: Adding a generic token field component #924
Conversation
youknowriad
added
the
General Interface
|
I do not think we should merge this without automated tests. Fortunately they already exist and we now have a PR to add component tests in this repository (#926). Can we / should we remove some of the unneeded functionality here like a token's |
Sure we can, I don't know if we should? Couldn't this be useful somewhere else? |
youknowriad
force-pushed
the
add/token-field-component
branch
from
c51f7fb
to
f33b026
Compare
|
Tests are failing on CI but passing locally but the error is weird. |
youknowriad
force-pushed
the
add/token-field-component
branch
from
f33b026
to
aaa7398
Compare
|
Tests are passing again. I'll appreciate some 👀 here. Thanks |
| @@ -30,6 +30,11 @@ $blue-medium-300: #66C6E4; | |||
| $blue-medium-200: #BFE7F3; | |||
| $blue-medium-100: #E5F5FA; | |||
|
|
|||
| // Alerts | |||
| $alert-yellow: #f0b849; | |||
There was a problem hiding this comment.
Odd spacing. Previous variables make no effort in aligning by space.
|
Oh maybe that's related to #957. Could do for a rebase then. |
youknowriad
force-pushed
the
add/token-field-component
branch
from
71ff59c
to
586aac8
Compare
|
@aduth Yep, it was related, should be fixed now. |
|
I forgot how slow these tests were, but we're now seeing Travis timeouts in unrelated branches because they take too long. We can bump timeouts, but I'd almost rather remove the tests before allowing ourselves to succumb to accepting slow tests. |
|
Even though these tests use |
In order to do #854 we need a generic Tags Input (with auto-complete...). Building such a component from scratch is not an easy task. So I shamelessly copied the component from Calypso and I adapted it to match our guidelines. It's a lot of code, so I'd appreciate help reviewing this component.
In order to test it. I'm using it in the sidebar with static values.
In a follow-up PR. I'll try to use it to update the post tags.